Skip to content

finish the task#15

Open
kawtharr25 wants to merge 1 commit into
TheCodePeople:mainfrom
kawtharr25:main
Open

finish the task#15
kawtharr25 wants to merge 1 commit into
TheCodePeople:mainfrom
kawtharr25:main

Conversation

@kawtharr25

Copy link
Copy Markdown

i finish the task

@ZehraaMA ZehraaMA left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You missed two other requirements, which are the isPopular and summary methods.

Comment thread object.js
},
addactor(newactor) {

this.actor = newactor

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The purpose of this function is to add a new actor to the actors list. However, what you did will replace the whole actors list with the new one. You can use either the concat method or the plus sign:
this.actors = this.actors.concat(", ", actorName);
this.actors = this.actors + ", " + actorName;

Comment thread object.js

}

let anothermovie=(...movie);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incorrect syntax:
let anothermovie={...movie};

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants